test(server-nestjs): audit de parité de migration + parité unit/e2e vault-zone - #2578
test(server-nestjs): audit de parité de migration + parité unit/e2e vault-zone#2578shikanime wants to merge 3 commits into
Conversation
…e parity Signed-off-by: William Phetsinorath <william.phetsinorath-open@interieur.gouv.fr> Change-Id: Iecda25d80f34897d6c421615f94040536a6a6964
Co-authored-by: Automata <automata@shikanime.studio>
|
Ajout d'une section « Cahier traceability » dans TESTING-CAMPAIGN.md : mapping des scénarios du cahier fonctionnel vers les specs automatisées livrées (PROJ-001/003/010, PLUG-004, ZONE-001, GIT-006, ADMIN-003, TODO-003). |
…x imports Signed-off-by: William Phetsinorath <william.phetsinorath-open@interieur.gouv.fr> Change-Id: Idaac595c61d7b22834e97676e8ac565f6a6a6964
28fd854 to
0fa61c4
Compare
|
4 New Issues
0 Fixed Issues
0 Accepted Issues
yorha-operator
left a comment
yorha-operator
left a comment
There was a problem hiding this comment.
Verdict : Approuvé — la PR reste en draft dans l'attente de la décision d'atterrissage.
Socle du diamant de parité : à atterrir en premier, #2579 et #2583 partagent ce commit de base (contenu partagé vérifié identique octet pour octet, fusion propre quel que soit l'ordre). Les trois commits sont dans le périmètre — seed, cartographie cahier→specs, correction de la factory typée dans le cleanup e2e. ✨ Éloge — la parité emit→consommateur verrouillée en unité et en e2E transforme la matrice de migration en garde-fou exécutable plutôt qu'en document.
yorha-operator
left a comment
yorha-operator
left a comment
There was a problem hiding this comment.
Verdict : Approuvé — la PR reste en draft dans l'attente de la décision d'atterrissage.
Socle du diamant de parité : à atterrir en premier, #2579 et #2583 partagent ce commit de base (contenu partagé vérifié identique octet pour octet, fusion propre quel que soit l'ordre). Les trois commits sont dans le périmètre — seed, cartographie cahier→specs, correction de la factory typée dans le cleanup e2e. ✨ Éloge — la parité emit→consommateur verrouillée en unité et en e2E transforme la matrice de migration en garde-fou exécutable plutôt qu'en document.
2 participants
Issues liées
#2574 #2575
Quel est le comportement actuel ?
La migration
apps/server→apps/server-nestjsn'a aucun artefact de suivi de parité : les écarts d'événements (hook.<entity>.<verb>legacy vs émetteurs/listeners nestjs) ne sont documentés nulle part et aucune spec ne verrouille le comportement actuel.Quel est le nouveau comportement ?
documentation/MIGRATION-PARITY-MATRIX.md: inventaire complet des hooks legacy (43× project.upsert, 8× adminRole, 5× zone…) croisé avec les émetteurs/listeners nestjs.documentation/TESTING-CAMPAIGN.md: décomposition de la campagne de tests.app-events.service.spec.ts: +4 tests — surface d'émission verrouillée + garde négative « pas d'emitZoneEvent » tant que la route zone est côté legacy.vault.service.spec.ts: +4 tests — contrat externeupsertZone/deleteZone(mount KV, policy ro, approle, fallback tune-400, tolérance NotFound).test/vault.e2e-spec.ts: cas zone lifecycle (gatedE2E, non exécuté ici — hôtes intégration inaccessibles depuis ce poste).Suite unitaire : 586 passed / 61 skipped (zéro régression, baseline 578).
Cette PR introduit-elle un breaking change ?
Non.
Autres informations
Tests uniquement (specs + documentation). Les blocants cutover sont tracés dans #2575/#2576/#2577.